45aae5fedf670f0fbe49b4a867c3bcf67f03bcab,clc/modules/walrus/src/main/java/edu/ucsb/eucalyptus/cloud/ws/WalrusManager.java,WalrusManager,getBucketAccessControlPolicy,#GetBucketAccessControlPolicyType#,564

Before Change


					try {
						//TODO: zhill - Modify this to handle invalid accounts and just skip them. This is just response creation, not authorization
						if (uId != null) {
							addPermission(grants, Accounts.lookupAccountById(uId), grantInfo);
						} else {
							addPermission(grants, grantInfo);
						}

After Change


							//Lots of work just to try to get the display Name of the userId.
							//TODO: zhill - Operations like this shouldn't have to hit the DB for every user, that can be in the grant record
							try {
								tmpAccnt = Accounts.lookupAccountById(uId);
								grantUser = new CanonicalUserType(tmpAccnt.getAccountNumber(), tmpAccnt.getName());
								tmpAccnt = null;								
							} catch(AuthException e) {
								LOG.debug(e,e);